guide: consolidate Exp Sharing intro #2711
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jorgeorpinel
force-pushed
the
guide/exps-sharing
branch
from
1db302c to
e17cb1d
Compare
jorgeorpinel
changed the title
jorgeorpinel
changed the title
jorgeorpinel
changed the title
jorgeorpinel
force-pushed
the
guide/exps-sharing
branch
from
e17cb1d to
58a6a0a
Compare
jorgeorpinel
force-pushed
the
guide/exps-sharing
branch
from
58a6a0a to
44a1614
Compare
jorgeorpinel
changed the title
|
@iesahin the main changes to review here are in content/docs/user-guide/experiment-management/sharing-experiments.md I thought we could do some small follow-up PRs like these to work on summarizing content. Feel free to contribute more changes or suggest alternatives beyond just reviewing. Thanks! |
jorgeorpinel
commented
Aug 15, 2021
content/docs/user-guide/experiment-management/sharing-experiments.md
Outdated
Show resolved
Hide resolved
jorgeorpinel
changed the title
jorgeorpinel
commented
Aug 15, 2021
daavoo
reviewed
Aug 18, 2021
There was a problem hiding this comment.
I think that we should point out that dvc exp push/dvc exp pull only work with SSH:
using git over ssh with the correct SSH keys configured is the only supported method for authenticated git push in dvc exp push
(from https://iterativeai.slack.com/archives/CB41NAL8H/p1628743292077800?thread_ts=1628690244.073500&cid=CB41NAL8H)
content/docs/user-guide/experiment-management/sharing-experiments.md
Outdated
Show resolved
Hide resolved
…nts.md Co-authored-by: David de la Iglesia Castro <[email protected]>
jorgeorpinel
commented
Aug 18, 2021
jorgeorpinel
commented
Aug 18, 2021
Comment on lines
-36
to
+31
| storage https://remote.dvc.org/get-started-experiments | ||
| storage s3://mybucket/my-dvc-store |
There was a problem hiding this comment.
p.s. I also changed the DVC remote from HTTP. I don't think you can push to an HTTP remote. Cc @iesahin
|
Looks like restyled is just not picking up broken files recently... I had to run |
jorgeorpinel
added a commit
that referenced
this pull request
Aug 20, 2021
* guide: summarize Sharing Exps intro * ref: link from exp push/pull to Exp Sharing guide * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: rename Exp Sharing sections * guide: summarize Exp Sharing examples * guide: link from Exp Mgmt index to Sharing * guide: ~~isolate~~ from link to Exp Sharing per #2711 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: David de la Iglesia Castro <[email protected]> * guide: mention only SSH Git URLs support exp sharing per #2711 (review) * guide: update dvc remote example in sharing exps * yarn format some files per https://app.circleci.com/pipelines/github/iterative/dvc.org/10086/workflows/9b1bf89f-a432-49f2-9a20-72fe77dd4102/jobs/10145 * prettier sharing-experiments.md * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: Casper da Costa-Luis <[email protected]> Co-authored-by: David de la Iglesia Castro <[email protected]> Co-authored-by: Casper da Costa-Luis <[email protected]>
jorgeorpinel
commented
Aug 24, 2021
7 tasks
2 tasks
jorgeorpinel
added a commit
that referenced
this pull request
Dec 23, 2021
* guide: split Experiments (index) into sub-pages * case: keep Persistent Exps in basic page * cases: keep Run-cache in basic Exps page * guide: edit Exp Mgmt index (intro) * guide: edit basic Exps page inc. persisting them and move run-cache to guide intro (index) * guide: rename DVC Exps, remove Org Exps page * guide: bash -> dvc in EM/Checkpoints * guide: fix exps link * guide: summarize Sharing Exps intro * ref: link from exp push/pull to Exp Sharing guide * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: rename Exp Sharing sections * guide: summarize Exp Sharing examples * guide: link from Exp Mgmt index to Sharing * guide: ~~isolate~~ from link to Exp Sharing per #2711 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: David de la Iglesia Castro <[email protected]> * guide: mention only SSH Git URLs support exp sharing per #2711 (review) * guide: update dvc remote example in sharing exps * yarn format some files per https://app.circleci.com/pipelines/github/iterative/dvc.org/10086/workflows/9b1bf89f-a432-49f2-9a20-72fe77dd4102/jobs/10145 * guide: consolidate Exp Sharing intro (#2711) * guide: summarize Sharing Exps intro * ref: link from exp push/pull to Exp Sharing guide * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: link from Exp Mgmt index to Sharing * guide: ~~isolate~~ from link to Exp Sharing per #2711 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: David de la Iglesia Castro <[email protected]> * guide: mention only SSH Git URLs support exp sharing per #2711 (review) * guide: update dvc remote example in sharing exps * yarn format some files per https://app.circleci.com/pipelines/github/iterative/dvc.org/10086/workflows/9b1bf89f-a432-49f2-9a20-72fe77dd4102/jobs/10145 Co-authored-by: David de la Iglesia Castro <[email protected]> * prettier sharing-experiments.md * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: Casper da Costa-Luis <[email protected]> * guide: roll back wrong files * guide: roll back Exp Mgmt index... * guide: link to Sharing Exps from index * guide: Listing exps on remotes per #2908 (review) * guide: don't mention Git here... per #2908 (review) * guide: clarify that git is needed for exps and sharing per #2908 (review) * guide: clarify note on Git requirement for DVC Exps per #2908 (review) * guide: simplify Sharing Exps intro (rel Git) per #2908 (review) * guide: rename exp list -r section * copy edit * cases: simplify note about requiring Git per #2908 (review) * guide: emoji for example in Sharing Exps per #2908 (comment) * guide: clarify note about Git-DVC repo required for Exps per #2908 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: another example emoji en Sharing Exps * Restyled by prettier (#2972) Co-authored-by: Restyled.io <[email protected]> * guide: list exps in Comparing guide, linked from Sharing per #2908 (comment) * guide: address feedback from #2908 (review) and below * guide: rephrase Git history exps org per #2908 (review) * guide:address Exp sharing feedback from from #2908 (review) and below * guide: update Git remote auth limitation wording per #2908 (comment) * guide: more copy edits on Exp Sharing and Comparing * guide: clarify `exp list` remote info per #2908 (review) * guide: un0hide exp sharing details per #2908 (review) * guide: move multi-exp share example to how-to per #2908 (review) * guide: simplify Exp Sharing intro, add diagram per should be focusing more on explaining (in simple terms, with diagrams) how it works * guide: fix SSH URLS link in Exp Sharing... * exp: roll back unrelated changes * guide: Git -> Git remote per #2908 (review) * guide: improve Sharing exp intro per #2908 (review) * exp push/pull: remove --remote and --jobs details from guide and ref descs. rel. #2908 (comment) * guide: remove Sharing Exps example per #2908 (comment) * guide: simplify Sharing Exps intro per #2908 (review) * guide: add exp pull to diagram in Sharing Exps per #2908 (comment) Co-authored-by: David de la Iglesia Castro <[email protected]> Co-authored-by: Casper da Costa-Luis <[email protected]> Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io <[email protected]>
iesahin
pushed a commit
that referenced
this pull request
Apr 11, 2022
* guide: split Experiments (index) into sub-pages * case: keep Persistent Exps in basic page * cases: keep Run-cache in basic Exps page * guide: edit Exp Mgmt index (intro) * guide: edit basic Exps page inc. persisting them and move run-cache to guide intro (index) * guide: rename DVC Exps, remove Org Exps page * guide: bash -> dvc in EM/Checkpoints * guide: fix exps link * guide: summarize Sharing Exps intro * ref: link from exp push/pull to Exp Sharing guide * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: rename Exp Sharing sections * guide: summarize Exp Sharing examples * guide: link from Exp Mgmt index to Sharing * guide: ~~isolate~~ from link to Exp Sharing per #2711 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: David de la Iglesia Castro <[email protected]> * guide: mention only SSH Git URLs support exp sharing per #2711 (review) * guide: update dvc remote example in sharing exps * yarn format some files per https://app.circleci.com/pipelines/github/iterative/dvc.org/10086/workflows/9b1bf89f-a432-49f2-9a20-72fe77dd4102/jobs/10145 * guide: consolidate Exp Sharing intro (#2711) * guide: summarize Sharing Exps intro * ref: link from exp push/pull to Exp Sharing guide * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: link from Exp Mgmt index to Sharing * guide: ~~isolate~~ from link to Exp Sharing per #2711 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: David de la Iglesia Castro <[email protected]> * guide: mention only SSH Git URLs support exp sharing per #2711 (review) * guide: update dvc remote example in sharing exps * yarn format some files per https://app.circleci.com/pipelines/github/iterative/dvc.org/10086/workflows/9b1bf89f-a432-49f2-9a20-72fe77dd4102/jobs/10145 Co-authored-by: David de la Iglesia Castro <[email protected]> * prettier sharing-experiments.md * Update content/docs/user-guide/experiment-management/sharing-experiments.md Co-authored-by: Casper da Costa-Luis <[email protected]> * guide: roll back wrong files * guide: roll back Exp Mgmt index... * guide: link to Sharing Exps from index * guide: Listing exps on remotes per #2908 (review) * guide: don't mention Git here... per #2908 (review) * guide: clarify that git is needed for exps and sharing per #2908 (review) * guide: clarify note on Git requirement for DVC Exps per #2908 (review) * guide: simplify Sharing Exps intro (rel Git) per #2908 (review) * guide: rename exp list -r section * copy edit * cases: simplify note about requiring Git per #2908 (review) * guide: emoji for example in Sharing Exps per #2908 (comment) * guide: clarify note about Git-DVC repo required for Exps per #2908 (review) * Update content/docs/user-guide/experiment-management/sharing-experiments.md * guide: another example emoji en Sharing Exps * Restyled by prettier (#2972) Co-authored-by: Restyled.io <[email protected]> * guide: list exps in Comparing guide, linked from Sharing per #2908 (comment) * guide: address feedback from #2908 (review) and below * guide: rephrase Git history exps org per #2908 (review) * guide:address Exp sharing feedback from from #2908 (review) and below * guide: update Git remote auth limitation wording per #2908 (comment) * guide: more copy edits on Exp Sharing and Comparing * guide: clarify `exp list` remote info per #2908 (review) * guide: un0hide exp sharing details per #2908 (review) * guide: move multi-exp share example to how-to per #2908 (review) * guide: simplify Exp Sharing intro, add diagram per should be focusing more on explaining (in simple terms, with diagrams) how it works * guide: fix SSH URLS link in Exp Sharing... * exp: roll back unrelated changes * guide: Git -> Git remote per #2908 (review) * guide: improve Sharing exp intro per #2908 (review) * exp push/pull: remove --remote and --jobs details from guide and ref descs. rel. #2908 (comment) * guide: remove Sharing Exps example per #2908 (comment) * guide: simplify Sharing Exps intro per #2908 (review) * guide: add exp pull to diagram in Sharing Exps per #2908 (comment) Co-authored-by: David de la Iglesia Castro <[email protected]> Co-authored-by: Casper da Costa-Luis <[email protected]> Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The idea behind this "consolidate" series of PRs (more to come) is to
a) summarize the explanations in this guide (without removing necessary info) and
b) to remove/bring details from the command reference (and link) per task #2 in #2548.
Follow up to #2618